Drop the last mentions of motion hint events
authorEmmanuele Bassi <ebassi@gnome.org>
Mon, 5 Feb 2018 13:52:46 +0000 (13:52 +0000)
committerEmmanuele Bassi <ebassi@gnome.org>
Mon, 5 Feb 2018 14:00:51 +0000 (14:00 +0000)
The GDK_POINTER_MOTION_HINT_MASK enumeration value is gone, but we're
still keeping around the "is_hint" field in GdkEventMotion, even though
every backend sets it to `false` — except for the core X11 device
manager.

gdk/gdkeventsprivate.h
gdk/quartz/gdkevents-quartz.c
gdk/wayland/gdkdevice-wayland.c
gdk/win32/gdkdevicemanager-win32.c
gdk/win32/gdkevents-win32.c
gdk/x11/gdkdevicemanager-core-x11.c
gdk/x11/gdkdevicemanager-xi2.c

index 41ffc5240c705e95e85fb1016292df279284f336..1daffc27b9bce05f35a6a093bb8957d8562415d7 100644 (file)
@@ -99,8 +99,6 @@ struct _GdkEventExpose
  * @state: (type GdkModifierType): a bit-mask representing the state of
  *   the modifier keys (e.g. Control, Shift and Alt) and the pointer
  *   buttons. See #GdkModifierType.
- * @is_hint: set to 1 if this event is just a hint, see the
- *   %GDK_POINTER_MOTION_HINT_MASK value of #GdkEventMask.
  * @device: the master device that the event originated from. Use
  * gdk_event_get_source_device() to get the slave device.
  * @x_root: the x coordinate of the pointer relative to the root of the
@@ -118,7 +116,6 @@ struct _GdkEventMotion
   gdouble y;
   gdouble *axes;
   guint state;
-  gint16 is_hint;
   GdkDeviceTool *tool;
   gdouble x_root, y_root;
   GList *history;
index 521c1a3d2e77a1956e08a983799a4cfda09dc71c..4544bc11ea319435726a29a3bbbae4af1d0e5f06 100644 (file)
@@ -316,7 +316,7 @@ get_event_mask_from_ns_event (NSEvent *nsevent)
     case NSOtherMouseUp:
       return GDK_BUTTON_RELEASE_MASK;
     case NSMouseMoved:
-      return GDK_POINTER_MOTION_MASK | GDK_POINTER_MOTION_HINT_MASK;
+      return GDK_POINTER_MOTION_MASK;
     case NSScrollWheel:
       /* Since applications that want button press events can get
        * scroll events on X11 (since scroll wheel events are really
@@ -324,20 +324,18 @@ get_event_mask_from_ns_event (NSEvent *nsevent)
        */
       return GDK_SCROLL_MASK | GDK_BUTTON_PRESS_MASK;
     case NSLeftMouseDragged:
-      return (GDK_POINTER_MOTION_MASK | GDK_POINTER_MOTION_HINT_MASK |
+      return (GDK_POINTER_MOTION_MASK |
              GDK_BUTTON_MOTION_MASK | GDK_BUTTON1_MOTION_MASK | 
              GDK_BUTTON1_MASK);
     case NSRightMouseDragged:
-      return (GDK_POINTER_MOTION_MASK | GDK_POINTER_MOTION_HINT_MASK |
+      return (GDK_POINTER_MOTION_MASK |
              GDK_BUTTON_MOTION_MASK | GDK_BUTTON3_MOTION_MASK | 
              GDK_BUTTON3_MASK);
     case NSOtherMouseDragged:
       {
        GdkEventMask mask;
 
-       mask = (GDK_POINTER_MOTION_MASK |
-               GDK_POINTER_MOTION_HINT_MASK |
-               GDK_BUTTON_MOTION_MASK);
+       mask = (GDK_POINTER_MOTION_MASK | GDK_BUTTON_MOTION_MASK);
 
        if (get_mouse_button_from_ns_event (nsevent) == 2)
          mask |= (GDK_BUTTON2_MOTION_MASK | GDK_BUTTON2_MOTION_MASK | 
@@ -535,7 +533,6 @@ generate_motion_event (GdkWindow *window)
   /* FIXME event->axes */
   event->motion.state = _gdk_quartz_events_get_current_keyboard_modifiers () |
                         _gdk_quartz_events_get_current_mouse_modifiers ();
-  event->motion.is_hint = FALSE;
   device_manager = GDK_QUARTZ_DEVICE_MANAGER_CORE (_gdk_device_manager);
   event->motion.device = device_manager->core_pointer;
   gdk_event_set_seat (event, gdk_device_get_seat (device_manager->core_pointer));
@@ -1070,7 +1067,6 @@ fill_motion_event (GdkWindow *window,
   /* FIXME event->axes */
   event->motion.state = get_keyboard_modifiers_from_ns_event (nsevent) |
                         _gdk_quartz_events_get_current_mouse_modifiers ();
-  event->motion.is_hint = FALSE;
   device_manager = GDK_QUARTZ_DEVICE_MANAGER_CORE (_gdk_device_manager);
   event->motion.device = device_manager->core_pointer;
   gdk_event_set_seat (event, gdk_device_get_seat (device_manager->core_pointer));
index 0946c32550502e2dd126ef98d51f668b65b1025b..7d8882f49312c9aa6ab2cebc90e6644d2120487d 100644 (file)
@@ -1482,7 +1482,6 @@ pointer_handle_motion (void              *data,
   event->motion.time = time;
   event->motion.axes = NULL;
   event->motion.state = device_get_modifiers (seat->master_pointer);
-  event->motion.is_hint = 0;
   gdk_event_set_display (event, seat->display);
 
   get_coordinates (seat->master_pointer,
@@ -3525,7 +3524,6 @@ tablet_tool_handle_motion (void                      *data,
   gdk_event_set_device_tool (event, tool->tool);
   event->motion.time = tablet->pointer_info.time;
   event->motion.state = device_get_modifiers (tablet->master);
-  event->motion.is_hint = FALSE;
   gdk_event_set_display (event, seat->display);
 
   get_coordinates (tablet->master,
index f6dbe60c4158c18418c4728c611ced873a0299ab..28507fba3836c6e4c3df6d38c08ca5c748906587 100644 (file)
@@ -1123,7 +1123,6 @@ gdk_input_other_event (GdkDisplay *display,
       else
         {
           event->motion.time = _gdk_win32_get_next_tick (msg->time);
-          event->motion.is_hint = FALSE;
           gdk_event_set_device (event, device_manager->core_pointer);
           gdk_event_set_source_device (event, GDK_DEVICE (source_device));
           gdk_event_set_seat (event, gdk_device_get_seat (device_manager->core_pointer));
index 59c9f5cc1f68c051c678af60892b93c36ca3bb42..83102adc97b5290b4bf699cbd1b7ffc01ab7293b 100644 (file)
@@ -815,10 +815,9 @@ _gdk_win32_print_event (const GdkEvent *event)
               event->expose.count);
       break;
     case GDK_MOTION_NOTIFY:
-      g_print ("(%.4g,%.4g) (%.4g,%.4g) %s",
+      g_print ("(%.4g,%.4g) (%.4g,%.4g)",
               event->motion.x, event->motion.y,
-              event->motion.x_root, event->motion.y_root,
-              event->motion.is_hint ? "HINT " : "");
+              event->motion.x_root, event->motion.y_root);
       print_event_state (event->motion.state);
       break;
     case GDK_BUTTON_PRESS:
@@ -2947,7 +2946,6 @@ gdk_event_translate (MSG  *msg,
          event->motion.y_root = current_root_y;
          event->motion.axes = NULL;
          event->motion.state = build_pointer_event_state (msg);
-         event->motion.is_hint = FALSE;
          gdk_event_set_device (event, device_manager_win32->core_pointer);
          gdk_event_set_source_device (event, device_manager_win32->system_pointer);
           gdk_event_set_seat (event, gdk_device_get_seat (device_manager_win32->core_pointer));
index 3062902c0431089d4121966e3d72b7578edf51f2..0845f2e5397935c76fb11536b5d796b707b8f686 100644 (file)
@@ -595,7 +595,6 @@ gdk_x11_device_manager_core_translate_event (GdkEventTranslator *translator,
       event->motion.y_root = (gdouble) xevent->xmotion.y_root / scale;
       event->motion.axes = NULL;
       event->motion.state = (GdkModifierType) xevent->xmotion.state;
-      event->motion.is_hint = xevent->xmotion.is_hint;
       gdk_event_set_device (event, device_manager->core_pointer);
 
       gdk_event_set_display (event, display);
index bd8fe7d6b19c387af4f2e8a40f3a02c18e450593..7f079b095869917f1fe9450a37ff7e3cad1dfece 100644 (file)
@@ -1643,9 +1643,6 @@ gdk_x11_device_manager_xi2_translate_event (GdkEventTranslator *translator,
 
         event->motion.state = _gdk_x11_device_xi2_translate_state (&xev->mods, &xev->buttons, &xev->group);
 
-        /* There doesn't seem to be motion hints in XI */
-        event->motion.is_hint = FALSE;
-
         event->motion.axes = translate_axes (device,
                                              event->motion.x,
                                              event->motion.y,